Skip to content

Rails 6.1: Fix the quoting of ActiveModel attributes #885

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Apr 19, 2021

Conversation

aidanharan
Copy link
Contributor

@aidanharan aidanharan commented Apr 19, 2021

PR fixes the TypeError: can't quote ActiveRecord::Relation::QueryAttribute errors in the tests.

This is an alternative fix to #883

@aidanharan aidanharan changed the title Fix quoting ActiveModel attribute Raisl 6.1: Fix when quoting ActiveModel::Attribute Apr 19, 2021
@aidanharan aidanharan changed the title Raisl 6.1: Fix when quoting ActiveModel::Attribute Raisl 6.1: Fix the quoting of ActiveModel attributes Apr 19, 2021
@aidanharan aidanharan changed the title Raisl 6.1: Fix the quoting of ActiveModel attributes Rails 6.1: Fix the quoting of ActiveModel attributes Apr 19, 2021
@aidanharan aidanharan marked this pull request as ready for review April 19, 2021 13:48
@wpolicarpo wpolicarpo merged commit bc1d46e into rails-sqlserver:main Apr 19, 2021
aidanharan pushed a commit to aidanharan/activerecord-sqlserver-adapter that referenced this pull request Apr 20, 2021
wpolicarpo pushed a commit that referenced this pull request Apr 20, 2021
Co-authored-by: Aidan Haran <aharan@fusioneer.com>
lavika pushed a commit to lavika/activerecord-sqlserver-adapter that referenced this pull request Sep 26, 2023
…#885)

* Fix when quoting ActiveModel::Attribute

* Updated changelog

Co-authored-by: Aidan Haran <aharan@fusioneer.com>
lavika pushed a commit to lavika/activerecord-sqlserver-adapter that referenced this pull request Sep 26, 2023
…-sqlserver#896)

Co-authored-by: Aidan Haran <aharan@fusioneer.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants